From: Marco Trevisan (Treviño) <mail@3v1n0.net>
Date: Thu, 13 Jun 2019 17:40:47 +0000 (+0200)
Subject: boxpointer: Don't trigger a relayout when updating arrow side
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~845^2^2~142
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=3d4ba028c4cc525da78ee966ce86a848aa09d606;p=gnome-shell.git

boxpointer: Don't trigger a relayout when updating arrow side

Changing the arrow side might need to reposition the boxpointer, however
if this happens during allocation, we don't need to trigger a new relayout since
we'd set the new allocation once _updateFlip's _reposition call is terminated,
otherwise if the position has changed, changing the boxpointer coordinates
will trigger a relayout anyways.

https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/576
---

diff --git a/js/ui/boxpointer.js b/js/ui/boxpointer.js
index 9939a5abf..2f4975a29 100644
--- a/js/ui/boxpointer.js
+++ b/js/ui/boxpointer.js
@@ -671,10 +671,6 @@ var BoxPointer = GObject.registerClass({
         if (this._arrowSide != arrowSide) {
             this._arrowSide = arrowSide;
             this._reposition(allocationBox);
-            Meta.later_add(Meta.LaterType.BEFORE_REDRAW, () => {
-                this.queue_relayout();
-                return false;
-            });
 
             this.emit('arrow-side-changed');
         }